Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add max_receive_msg_size parameters. #230

Merged
merged 2 commits into from
Mar 27, 2024
Merged

Conversation

xiaoxichen
Copy link
Contributor

max_receive_msg_size default to 4M, which might be too small for some usecases, allow this to be reconfigurable.

@raakella1 raakella1 requested a review from yamingk March 21, 2024 20:11
max_receive_msg_size default to 4M, which might be too small
for some usecases, allow this to be reconfigurable.

Signed-off-by: Xiaoxi Chen <[email protected]>
Signed-off-by: Xiaoxi Chen <[email protected]>
@codecov-commenter
Copy link

Codecov Report

Merging #230 (dfacbfd) into master (c3b8844) will increase coverage by 63.70%.
The diff coverage is 66.66%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #230       +/-   ##
===========================================
+ Coverage        0   63.70%   +63.70%     
===========================================
  Files           0       72       +72     
  Lines           0     4405     +4405     
  Branches        0      555      +555     
===========================================
+ Hits            0     2806     +2806     
- Misses          0     1349     +1349     
- Partials        0      250      +250     
Components Coverage Δ
AuthManager 77.77% <ø> (∅)
Cache 25.23% <ø> (∅)
FDS 71.01% <ø> (∅)
FileWatcher 56.25% <ø> (∅)
Flip 65.34% <ø> (∅)
gRPC 77.04% <66.66%> (∅)
Logging 30.18% <ø> (∅)
Metrics 80.20% <ø> (∅)
Options 100.00% <ø> (∅)
Setting 56.79% <ø> (∅)
StatusObject 73.83% <ø> (∅)
Utility 82.71% <ø> (∅)
Version 95.83% <ø> (∅)

@hkadayam hkadayam merged commit 370c772 into eBay:master Mar 27, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants